Skip to content

[azquery] Change MetricsClientQueryResourceOptions.Aggregation from type *string to []*AggregationType#19981

Merged
gracewilcox merged 12 commits intoAzure:mainfrom
gracewilcox:azquery/release-updates
Feb 8, 2023
Merged

[azquery] Change MetricsClientQueryResourceOptions.Aggregation from type *string to []*AggregationType#19981
gracewilcox merged 12 commits intoAzure:mainfrom
gracewilcox:azquery/release-updates

Conversation

@gracewilcox
Copy link
Copy Markdown
Member

Updates Aggregation to use a slice of the already generated AggregationType instead of a comma separated string.

@gracewilcox gracewilcox added Client This issue points to a problem in the data-plane of the library. Monitor - Query labels Feb 8, 2023
Comment thread sdk/monitor/azquery/example_test.go Outdated
Comment thread sdk/monitor/azquery/metrics_client_test.go Outdated
Comment thread sdk/monitor/azquery/assets.json
@gracewilcox gracewilcox merged commit afa7fca into Azure:main Feb 8, 2023
@gracewilcox gracewilcox deleted the azquery/release-updates branch February 8, 2023 23:51
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Query labels Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. Monitor Monitor, Monitor Ingestion, Monitor Query

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants